gdk: Drop GdkGrabOwnership
authorCarlos Garnacho <carlosg@gnome.org>
Mon, 22 Jun 2020 21:54:46 +0000 (23:54 +0200)
committerCarlos Garnacho <carlosg@gnome.org>
Tue, 23 Jun 2020 21:42:53 +0000 (23:42 +0200)
We no longer expose such low level tweaks, this is essentially
unused.

gdk/broadway/gdksurface-broadway.c
gdk/gdkdevice.c
gdk/gdkdisplay.c
gdk/gdkdisplayprivate.h
gdk/gdkinternals.h
gdk/gdkseatdefault.c
gdk/gdksurface.c
gdk/wayland/gdkdevice-wayland.c
gdk/win32/gdkdrag-win32.c
gdk/win32/gdksurface-win32.c
gdk/x11/gdkdrag-x11.c

index f8702247f6651a76a7270f210fe053527e7f8008..5b278ce7c3746127426116fccf3a596c90641793 100644 (file)
@@ -1107,13 +1107,12 @@ create_moveresize_surface (MoveResizeData *mv_resize,
 
   G_GNUC_BEGIN_IGNORE_DEPRECATIONS;
   status = gdk_device_grab (pointer,
-                             mv_resize->moveresize_emulation_surface,
-                             GDK_OWNERSHIP_APPLICATION,
-                             FALSE,
-                             GDK_BUTTON_RELEASE_MASK |
-                             GDK_POINTER_MOTION_MASK,
-                             NULL,
-                             timestamp);
+                            mv_resize->moveresize_emulation_surface,
+                            FALSE,
+                            GDK_BUTTON_RELEASE_MASK |
+                            GDK_POINTER_MOTION_MASK,
+                            NULL,
+                            timestamp);
   G_GNUC_END_IGNORE_DEPRECATIONS;
 
   if (status != GDK_GRAB_SUCCESS)
index a18d4d76b55bdfe07d7eda775fff881722235846..29a1fcab47b9d096da7ed1dcd13af3c588c2c357 100644 (file)
@@ -924,7 +924,6 @@ get_native_grab_event_mask (GdkEventMask grab_mask)
 GdkGrabStatus
 gdk_device_grab (GdkDevice        *device,
                  GdkSurface       *surface,
-                 GdkGrabOwnership  grab_ownership,
                  gboolean          owner_events,
                  GdkEventMask      event_mask,
                  GdkCursor        *cursor,
@@ -958,7 +957,6 @@ gdk_device_grab (GdkDevice        *device,
       _gdk_display_add_device_grab (display,
                                     device,
                                     surface,
-                                    grab_ownership,
                                     owner_events,
                                     event_mask,
                                     serial,
index db13ea52c674d0ec2d1432eae58a3bcf061012e5..b06f412caf872f8389be0b48394a755f18e90cc0 100644 (file)
@@ -521,7 +521,6 @@ GdkDeviceGrabInfo *
 _gdk_display_add_device_grab (GdkDisplay       *display,
                               GdkDevice        *device,
                               GdkSurface        *surface,
-                              GdkGrabOwnership  grab_ownership,
                               gboolean          owner_events,
                               GdkEventMask      event_mask,
                               unsigned long     serial_start,
@@ -540,7 +539,6 @@ _gdk_display_add_device_grab (GdkDisplay       *display,
   info->event_mask = event_mask;
   info->time = time;
   info->implicit = implicit;
-  info->ownership = grab_ownership;
 
   grabs = g_hash_table_lookup (display->device_grabs, device);
 
@@ -816,61 +814,6 @@ _gdk_display_end_device_grab (GdkDisplay *display,
   return FALSE;
 }
 
-/* Returns TRUE if device events are not blocked by any grab */
-gboolean
-_gdk_display_check_grab_ownership (GdkDisplay *display,
-                                   GdkDevice  *device,
-                                   gulong      serial)
-{
-  GHashTableIter iter;
-  gpointer key, value;
-  GdkGrabOwnership higher_ownership, device_ownership;
-  gboolean device_is_keyboard;
-
-  g_hash_table_iter_init (&iter, display->device_grabs);
-  higher_ownership = device_ownership = GDK_OWNERSHIP_NONE;
-  device_is_keyboard = (gdk_device_get_source (device) == GDK_SOURCE_KEYBOARD);
-
-  while (g_hash_table_iter_next (&iter, &key, &value))
-    {
-      GdkDeviceGrabInfo *grab;
-      GdkDevice *dev;
-      GList *grabs;
-
-      dev = key;
-      grabs = value;
-      grabs = grab_list_find (grabs, serial);
-
-      if (!grabs)
-        continue;
-
-      /* Discard device if it's not of the same type */
-      if ((device_is_keyboard && gdk_device_get_source (dev) != GDK_SOURCE_KEYBOARD) ||
-          (!device_is_keyboard && gdk_device_get_source (dev) == GDK_SOURCE_KEYBOARD))
-        continue;
-
-      grab = grabs->data;
-
-      if (dev == device)
-        device_ownership = grab->ownership;
-      else
-        {
-          if (grab->ownership > higher_ownership)
-            higher_ownership = grab->ownership;
-        }
-    }
-
-  if (higher_ownership > device_ownership)
-    {
-      /* There's a higher priority ownership
-       * going on for other device(s)
-       */
-      return FALSE;
-    }
-
-  return TRUE;
-}
-
 GdkPointerSurfaceInfo *
 _gdk_display_get_pointer_info (GdkDisplay *display,
                                GdkDevice  *device)
index d29cf2ebed1900f291f2846d343ce58a37cd2725..8a50d3cd3a5d62f8b12889dd6e13198e51f1ab18 100644 (file)
@@ -45,7 +45,6 @@ typedef struct
   gulong serial_end; /* exclusive, i.e. not active on serial_end */
   guint event_mask;
   guint32 time;
-  GdkGrabOwnership ownership;
 
   guint activated : 1;
   guint implicit_ungrab : 1;
@@ -187,7 +186,6 @@ GdkDeviceGrabInfo * _gdk_display_get_last_device_grab (GdkDisplay *display,
 GdkDeviceGrabInfo * _gdk_display_add_device_grab      (GdkDisplay       *display,
                                                        GdkDevice        *device,
                                                        GdkSurface        *surface,
-                                                       GdkGrabOwnership  grab_ownership,
                                                        gboolean          owner_events,
                                                        GdkEventMask      event_mask,
                                                        gulong            serial_start,
@@ -201,9 +199,6 @@ gboolean            _gdk_display_end_device_grab      (GdkDisplay       *display
                                                        gulong            serial,
                                                        GdkSurface        *if_child,
                                                        gboolean          implicit);
-gboolean            _gdk_display_check_grab_ownership (GdkDisplay       *display,
-                                                       GdkDevice        *device,
-                                                       gulong            serial);
 GdkPointerSurfaceInfo * _gdk_display_get_pointer_info  (GdkDisplay       *display,
                                                        GdkDevice        *device);
 void                _gdk_display_pointer_info_foreach (GdkDisplay       *display,
index b26aac73ebac3f56def7dd7cbe9094bf65e8b32d..c42728573e4567959baac0803a62901ebe6e418c 100644 (file)
@@ -229,21 +229,6 @@ typedef enum
   GDK_GRAB_FAILED          = 5
 } GdkGrabStatus;
 
-/**
- * GdkGrabOwnership:
- * @GDK_OWNERSHIP_NONE: All other devices’ events are allowed.
- * @GDK_OWNERSHIP_SURFACE: Other devices’ events are blocked for the grab surface.
- * @GDK_OWNERSHIP_APPLICATION: Other devices’ events are blocked for the whole application.
- *
- * Defines how device grabs interact with other devices.
- */
-typedef enum
-{
-  GDK_OWNERSHIP_NONE,
-  GDK_OWNERSHIP_SURFACE,
-  GDK_OWNERSHIP_APPLICATION
-} GdkGrabOwnership;
-
 typedef enum
 {
   GDK_EXPOSURE_MASK             = 1 << 1,
@@ -274,7 +259,6 @@ typedef enum
 
 GdkGrabStatus gdk_device_grab (GdkDevice        *device,
                                GdkSurface        *surface,
-                               GdkGrabOwnership  grab_ownership,
                                gboolean          owner_events,
                                GdkEventMask      event_mask,
                                GdkCursor        *cursor,
index ef0fe299d46ab1e3553fb1b1fa8b75402339ca62..822c8d2e64e5edaa5662756f0716c2dcfcc6a9bd 100644 (file)
@@ -149,7 +149,7 @@ gdk_seat_default_grab (GdkSeat                *seat,
         pointer_evmask |= TOUCH_EVENTS;
 
       status = gdk_device_grab (priv->logical_pointer, surface,
-                                GDK_OWNERSHIP_NONE, owner_events,
+                                owner_events,
                                 pointer_evmask, cursor,
                                 evtime);
     }
@@ -158,7 +158,7 @@ gdk_seat_default_grab (GdkSeat                *seat,
       capabilities & GDK_SEAT_CAPABILITY_KEYBOARD)
     {
       status = gdk_device_grab (priv->logical_keyboard, surface,
-                                GDK_OWNERSHIP_NONE, owner_events,
+                                owner_events,
                                 KEYBOARD_EVENTS, cursor,
                                 evtime);
 
index 93662b8b6383db3f1d0d3e70c7b59726e05a6817..6eac19ac37feab3344030102e62f8b52ea44af25 100644 (file)
@@ -2245,13 +2245,6 @@ _gdk_windowing_got_event (GdkDisplay *display,
         }
 
       _gdk_display_device_grab_update (display, device, source_device, serial);
-
-      if (!_gdk_display_check_grab_ownership (display, device, serial))
-        {
-          /* Device events are blocked by another device grab */
-          unlink_event = TRUE;
-          goto out;
-        }
     }
 
   event_surface = gdk_event_get_surface (event);
@@ -2274,7 +2267,6 @@ _gdk_windowing_got_event (GdkDisplay *display,
           _gdk_display_add_device_grab (display,
                                         device,
                                         event_surface,
-                                        GDK_OWNERSHIP_NONE,
                                         FALSE,
                                         GDK_ALL_EVENTS_MASK,
                                         serial,
index b18006b0c61ada5a570a89cda29c1fd148d2924a..d9b3ee8fa5e9f3062ec6decb64242768f64aac71 100644 (file)
@@ -4582,7 +4582,6 @@ gdk_wayland_seat_grab (GdkSeat                *seat,
       _gdk_display_add_device_grab (display,
                                     wayland_seat->logical_pointer,
                                     surface,
-                                    GDK_OWNERSHIP_NONE,
                                     owner_events,
                                     GDK_ALL_EVENTS_MASK,
                                     _gdk_display_get_next_serial (display),
@@ -4603,7 +4602,6 @@ gdk_wayland_seat_grab (GdkSeat                *seat,
       _gdk_display_add_device_grab (display,
                                     wayland_seat->logical_touch,
                                     surface,
-                                    GDK_OWNERSHIP_NONE,
                                     owner_events,
                                     GDK_ALL_EVENTS_MASK,
                                     _gdk_display_get_next_serial (display),
@@ -4620,7 +4618,6 @@ gdk_wayland_seat_grab (GdkSeat                *seat,
       _gdk_display_add_device_grab (display,
                                     wayland_seat->logical_keyboard,
                                     surface,
-                                    GDK_OWNERSHIP_NONE,
                                     owner_events,
                                     GDK_ALL_EVENTS_MASK,
                                     _gdk_display_get_next_serial (display),
@@ -4646,7 +4643,6 @@ gdk_wayland_seat_grab (GdkSeat                *seat,
           _gdk_display_add_device_grab (display,
                                         tablet->logical_device,
                                         surface,
-                                        GDK_OWNERSHIP_NONE,
                                         owner_events,
                                         GDK_ALL_EVENTS_MASK,
                                         _gdk_display_get_next_serial (display),
index a893401bd8332fbb4c2193df528e5b36ae00578d..04169bedeb3dc1858cd78bcc72304a3f8d811267 100644 (file)
@@ -2048,7 +2048,7 @@ gdk_win32_drag_set_cursor (GdkDrag *drag,
       G_GNUC_BEGIN_IGNORE_DEPRECATIONS;
       gdk_device_grab (gdk_seat_get_pointer (drag_win32->grab_seat),
                        drag_win32->grab_surface,
-                       GDK_OWNERSHIP_APPLICATION, FALSE,
+                       FALSE,
                        GDK_POINTER_MOTION_MASK | GDK_BUTTON_RELEASE_MASK,
                        cursor, GDK_CURRENT_TIME);
       G_GNUC_END_IGNORE_DEPRECATIONS;
index 5edb59e9a837538b62072cce4c20b86b7b065914..3b26a475ab0e5ff5fa69c0761465f8c5263e2638 100644 (file)
@@ -3762,7 +3762,7 @@ setup_drag_move_resize_context (GdkSurface                   *window,
    * This is why we first do the grab, *then* set the op.
    */
   gdk_device_grab (device, pointer_window,
-                   GDK_OWNERSHIP_NONE, FALSE,
+                   FALSE,
                    GDK_ALL_EVENTS_MASK,
                    context->cursor,
                    timestamp);
index 7e730e799311bdd0220e530ac8a16f6377114380..5a08c3af64a33d834448154ce07b52b438b5fa9d 100644 (file)
@@ -2077,7 +2077,7 @@ gdk_x11_drag_set_cursor (GdkDrag   *drag,
       G_GNUC_BEGIN_IGNORE_DEPRECATIONS;
       gdk_device_grab (gdk_seat_get_pointer (x11_drag->grab_seat),
                        x11_drag->ipc_surface,
-                       GDK_OWNERSHIP_APPLICATION, FALSE,
+                       FALSE,
                        GDK_POINTER_MOTION_MASK | GDK_BUTTON_RELEASE_MASK,
                        cursor, GDK_CURRENT_TIME);
       G_GNUC_END_IGNORE_DEPRECATIONS;